Skip to content

Add progression from small to large constellation for Star Map#48

Merged
K-Meech merged 10 commits intomainfrom
km/star-map-progression
Feb 5, 2026
Merged

Add progression from small to large constellation for Star Map#48
K-Meech merged 10 commits intomainfrom
km/star-map-progression

Conversation

@K-Meech
Copy link
Collaborator

@K-Meech K-Meech commented Jan 30, 2026

For #17 (see new info on adaptive difficulty from David on this issue)

Adds:

  • Game starts with a small constellation (5 stars)
  • After winning two games in a row with the max sequence length (=5), upgrades to the large constellation
  • Removed the set number of guessed sequences required to win. Now the game ends on reaching the max sequence length (= number of stars in constellation) OR after guessing a sequence correctly after having the length downgraded. This allows longer sequences to be shown (up to a length of 10)
  • Fixed issue with star selection, noted on this comment

@K-Meech K-Meech marked this pull request as ready for review February 4, 2026 14:13
Copy link
Member

@thompson318 thompson318 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works for me and the code looks good. I think it will need a bit more play testing with children of the right age to refine the game play.

@K-Meech
Copy link
Collaborator Author

K-Meech commented Feb 5, 2026

Thanks @thompson318 - I'll go ahead and merge this. I agree - will definitely need some refinement after play testing. Hopefully David will have time to try it quickly this week + give some suggestions.

@K-Meech K-Meech merged commit fa060d6 into main Feb 5, 2026
1 check passed
@K-Meech K-Meech deleted the km/star-map-progression branch February 5, 2026 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants